Surface UHD 4.10 'mtu' stream argument#67
Open
gretel wants to merge 1 commit into
Open
Conversation
UHD 4.10 added two new stream/device arguments: - 'mtu' (stream-arg) -- override the link-detected MTU - 'force_mtu' (device-arg) -- skip MTU detection entirely Both already flow through SoapyUHD without code change: setupStream() passes its kwargs verbatim into stream_args_t::args via kwargsToDict(), and make_uhd() does the same for device_args. They work today on UHD 4.10 -- they just aren't surfaced in getStreamArgsInfo(). Pothos GUI / SoapySDRUtil --probe / Sonneman / etc. enumerate the ArgInfo list when populating UI knobs and hint dropdowns. Without an ArgInfo entry, users have to know the magic key by reading UHD docs and type it manually. Adding one ArgInfo struct surfaces 'mtu' in every Soapy front-end automatically. Use case: remote streaming over WAN where path-MTU discovery fails (common with X410/X310 over consumer routers), or local network topologies that interfere with auto-detection. Guard with '#if UHD_VERSION >= 4100000' so the option only shows on UHD versions that actually honor it. The matching 'force_mtu' device arg already flows through; documenting it requires no ArgInfo (we don't expose getDeviceArgsInfo at the Soapy layer). No code change for the runtime path -- the kwargs already pipe to stream_args_t::args. This commit is pure metadata advertisement. Signed-off-by: Tom Hensel <code@jitter.eu>
This file contains hidden or bidirectional Unicode text that may be interpreted or compiled differently than what appears below. To review, open the file in an editor that reveals hidden Unicode characters.
Learn more about bidirectional Unicode characters
Sign up for free
to join this conversation on GitHub.
Already have an account?
Sign in to comment
Add this suggestion to a batch that can be applied as a single commit.This suggestion is invalid because no changes were made to the code.Suggestions cannot be applied while the pull request is closed.Suggestions cannot be applied while viewing a subset of changes.Only one suggestion per line can be applied in a batch.Add this suggestion to a batch that can be applied as a single commit.Applying suggestions on deleted lines is not supported.You must change the existing code in this line in order to create a valid suggestion.Outdated suggestions cannot be applied.This suggestion has been applied or marked resolved.Suggestions cannot be applied from pending reviews.Suggestions cannot be applied on multi-line comments.Suggestions cannot be applied while the pull request is queued to merge.Suggestion cannot be applied right now. Please check back later.
Surface UHD 4.10 'mtu' stream argument
Changed
mtuArgInfo entry toSoapyUHDDevice::getStreamArgsInfo().Why
UHD 4.10 added an
mtustream-arg that overrides the link-detected MTU. The arg already passes through transparently viakwargsToDict(); this PR makes it discoverable so Pothos / SoapySDR GUIs andSoapySDRUtil --probeshow the knob.